Skip to content

Conversation

jsdt
Copy link
Contributor

@jsdt jsdt commented Sep 24, 2025

Description of Changes

This exposes client credentials in reducer calls for rust.

API and ABI breaking changes

API Changes:

The main API change is the addition of AuthCtx and the sender_auth in ReducerContext. This also adds JwtClaims, which has some helpers for getting commonly used claims.

ABI Changes:

This adds one new functions get_jwt. This uses st_connection_credentials to look up the credentials associated with a connection id.

This adds ABI version 10.2.

Expected complexity level and risk

  1. This adds new ABI functions

Testing

I've done some manual testing with modified versions of the quickstart. We should add some examples that use the new API.

@jsdt jsdt requested a review from gefjon September 24, 2025 20:28
@Centril Centril self-requested a review September 25, 2025 09:59
@bfops bfops added the release-any To be landed in any release window label Sep 29, 2025
Copy link
Contributor

@Centril Centril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there were also stuff from my previous review that was not addressed.

@jsdt jsdt requested a review from Centril October 14, 2025 16:48
Copy link
Contributor

@Centril Centril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have some improvements atop of this PR, primarily wrt. docs, but this looks good to merge.

Copy link
Contributor

@gefjon gefjon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think I'm a code-owner of anything touched by this PR anymore, but it looks good.

Copy link
Collaborator

@bfops bfops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Signing off on my code-owned files under crates/cli. Afaict they're unrelated cleanups, but innocuous enough.

@jsdt jsdt enabled auto-merge October 17, 2025 18:31
@jsdt jsdt added this pull request to the merge queue Oct 17, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Oct 17, 2025
@bfops bfops enabled auto-merge October 17, 2025 19:34
@bfops bfops added this pull request to the merge queue Oct 17, 2025
Merged via the queue into master with commit dbc49b1 Oct 17, 2025
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-any To be landed in any release window

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants